Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

List all the fixable rules in CLI via --list-rules #3737

Merged
merged 7 commits into from
Sep 20, 2023

Conversation

shatakshiiii
Copy link
Contributor

@shatakshiiii shatakshiiii commented Sep 14, 2023

This PR adds the autofix tag in the --list-rules output, for all the applicable rules.

To see the rules under autofix, do: ansible-lint --list-rules or ansible-lint -L
or ansible-lint -L | grep autofix

Related: #3624, #3634

@shatakshiiii shatakshiiii temporarily deployed to ack September 14, 2023 08:16 — with GitHub Actions Inactive
@shatakshiiii shatakshiiii temporarily deployed to ack September 14, 2023 08:16 — with GitHub Actions Inactive
@shatakshiiii shatakshiiii temporarily deployed to ack September 15, 2023 06:05 — with GitHub Actions Inactive
@shatakshiiii shatakshiiii temporarily deployed to ack September 15, 2023 06:11 — with GitHub Actions Inactive
@shatakshiiii shatakshiiii self-assigned this Sep 15, 2023
@ssbarnea
Copy link
Member

Please use autofix label to mark these rules.

@shatakshiiii shatakshiiii temporarily deployed to ack September 20, 2023 08:44 — with GitHub Actions Inactive
@shatakshiiii shatakshiiii temporarily deployed to ack September 20, 2023 10:47 — with GitHub Actions Inactive
@shatakshiiii shatakshiiii changed the title [WIP]: List all the transform rules via CLI List all the fixable rules in CLI via --list-rules Sep 20, 2023
@ssbarnea ssbarnea marked this pull request as ready for review September 20, 2023 13:45
@ssbarnea ssbarnea requested a review from a team as a code owner September 20, 2023 13:45
@ssbarnea ssbarnea requested a review from a team September 20, 2023 13:45
@ssbarnea ssbarnea added bug and removed enhancement labels Sep 20, 2023
@ssbarnea ssbarnea temporarily deployed to ack September 20, 2023 13:47 — with GitHub Actions Inactive
@ssbarnea ssbarnea temporarily deployed to ack September 20, 2023 13:47 — with GitHub Actions Inactive
@ssbarnea ssbarnea merged commit b4b8b2c into ansible:main Sep 20, 2023
21 checks passed
@shatakshiiii shatakshiiii deleted the list_transform_rules branch September 20, 2023 16:04
nrdufour added a commit to nrdufour/home-ops that referenced this pull request Oct 3, 2023
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [ansible-lint](https://github.com/ansible/ansible-lint) ([changelog](https://github.com/ansible/ansible-lint/releases)) | patch | `==6.20.0` -> `==6.20.1` |

---

### Release Notes

<details>
<summary>ansible/ansible-lint (ansible-lint)</summary>

### [`v6.20.1`](https://github.com/ansible/ansible-lint/releases/tag/v6.20.1)

[Compare Source](ansible/ansible-lint@v6.20.0...v6.20.1)

#### Bugfixes

-   Fix detection of play roles vars missing prefix ([#&#8203;3765](ansible/ansible-lint#3765)) [@&#8203;cavcrosby](https://github.com/cavcrosby)
-   Prune reported errors after autofix ([#&#8203;3774](ansible/ansible-lint#3774)) [@&#8203;cidrblock](https://github.com/cidrblock)
-   `--fix` will now default to `all` ([#&#8203;3777](ansible/ansible-lint#3777)) [@&#8203;ajinkyau](https://github.com/ajinkyau)
-   Avoid false error caused by TypeError with jinja rule ([#&#8203;3773](ansible/ansible-lint#3773)) [@&#8203;ssbarnea](https://github.com/ssbarnea)
-   Sanity Rule Ignore Additions and Updates ([#&#8203;3766](ansible/ansible-lint#3766)) [@&#8203;alisonlhart](https://github.com/alisonlhart)
-   Update rulebook schema (name is now required) ([#&#8203;3776](ansible/ansible-lint#3776)) [@&#8203;ssbarnea](https://github.com/ssbarnea)
-   Expose rules that have autofix capability in docs ([#&#8203;3770](ansible/ansible-lint#3770)) [@&#8203;shatakshiiii](https://github.com/shatakshiiii)
-   List all the fixable rules in CLI via `--list-rules` ([#&#8203;3737](ansible/ansible-lint#3737)) [@&#8203;shatakshiiii](https://github.com/shatakshiiii)
-   Include level in SARIF results ([#&#8203;3758](ansible/ansible-lint#3758)) [@&#8203;ssbarnea](https://github.com/ssbarnea)
-   Meta runtime supported version update ([#&#8203;3755](ansible/ansible-lint#3755)) [@&#8203;alisonlhart](https://github.com/alisonlhart)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjEiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjEiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

Reviewed-on: https://git.home/nrdufour/home-ops/pulls/120
Co-authored-by: Renovate <renovate@ptinem.io>
Co-committed-by: Renovate <renovate@ptinem.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants